Skip to content

[None][chore] Update flashinfer-python from 0.6.12 to 0.6.14#15869

Open
yihwang-nv wants to merge 2 commits into
NVIDIA:mainfrom
yihwang-nv:yihwang-nv/update_flashinfer_0.6.14
Open

[None][chore] Update flashinfer-python from 0.6.12 to 0.6.14#15869
yihwang-nv wants to merge 2 commits into
NVIDIA:mainfrom
yihwang-nv:yihwang-nv/update_flashinfer_0.6.14

Conversation

@yihwang-nv

@yihwang-nv yihwang-nv commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Bump flashinfer-python from 0.6.12 to 0.6.14 (latest stable)
  • Updated version pins in requirements.txt, security_scanning/pyproject.toml, and ATTRIBUTIONS-Python.md
  • security_scanning/poetry.lock intentionally left untouched (regenerated separately by maintainers)
  • Version-gated logic in tensorrt_llm/_torch/speculative/interface.py (>= 0.6.4) still satisfied — no code change needed

Test plan

  • pip install -r requirements.txt installs successfully
  • pytest tests/unittest/_torch/flashinfer/ -v
  • pytest tests/unittest/_torch/attention/test_flashinfer_attention.py -v
  • CI pre-merge passes

Summary by CodeRabbit

  • Chores
    • Updated the pinned flashinfer-python version from 0.6.12 to 0.6.14 across project dependency and attribution records.
    • Kept dependency references aligned in the main requirements and security-scanning configuration.

Bump flashinfer-python dependency to the latest stable release.
Updated version pins in requirements.txt, security_scanning/pyproject.toml,
and ATTRIBUTIONS-Python.md.

Signed-off-by: Yihan Wang <yihwang@nvidia.com>
@yihwang-nv yihwang-nv requested a review from a team as a code owner July 2, 2026 05:34
@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request bumps the pinned version of the flashinfer-python dependency from 0.6.12 to 0.6.14 in requirements.txt and security_scanning/pyproject.toml, and updates the corresponding attribution entry in ATTRIBUTIONS-Python.md.

Changes

Dependency version bump

Layer / File(s) Summary
flashinfer-python version update
requirements.txt, security_scanning/pyproject.toml, ATTRIBUTIONS-Python.md
Updates the pinned flashinfer-python dependency version from 0.6.12 to 0.6.14 and syncs the corresponding attribution entry.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Related issues: None mentioned in the provided context.

Related PRs: None mentioned in the provided context.

Suggested labels: dependencies

Suggested reviewers: None specified.


🐰 A tiny hop, a version's change,
From twelve to fourteen, nothing strange,
Attribution now aligned so neat,
A quiet bump, a task complete!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title matches the PR's main change and follows the repository's required [None][chore] format.
Description check ✅ Passed The PR description includes a clear summary and test plan, covering the main required sections.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@security_scanning/pyproject.toml`:
- Line 57: The dependency pin in security_scanning/pyproject.toml was updated to
flashinfer-python 0.6.14, but security_scanning/poetry.lock still reflects
0.6.12. Regenerate the lockfile so the Poetry lock state matches the updated
dependency constraint, and ensure the flashinfer-python entry in poetry.lock
resolves to 0.6.14.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: dee9fe85-3f90-457f-a3bf-9f009af9be78

📥 Commits

Reviewing files that changed from the base of the PR and between c81fa33 and 9e3e11a.

📒 Files selected for processing (3)
  • ATTRIBUTIONS-Python.md
  • requirements.txt
  • security_scanning/pyproject.toml

Comment thread security_scanning/pyproject.toml
@yihwang-nv

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast --add-multi-gpu-test

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57128 [ run ] triggered by Bot. Commit: 9e3e11a Link to invocation

@yuxianq yuxianq self-requested a review July 3, 2026 03:18
@yihwang-nv

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast --add-multi-gpu-test

1 similar comment
@yihwang-nv

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast --add-multi-gpu-test

Signed-off-by: Yihan Wang <yihwang@nvidia.com>
@yihwang-nv

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast --add-multi-gpu-test

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57366 [ run ] triggered by Bot. Commit: 49d3504 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57128 [ run ] completed with state ABORTED. Commit: 9e3e11a

Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57366 [ run ] completed with state SUCCESS. Commit: 49d3504
/LLM/main/L0_MergeRequest_PR pipeline #46117 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@nv-guomingz

Copy link
Copy Markdown
Collaborator

/bot run --disable-fail-fast --add-multi-gpu-test

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57541 [ run ] triggered by Bot. Commit: 49d3504 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57541 [ run ] completed with state SUCCESS. Commit: 49d3504
/LLM/main/L0_MergeRequest_PR pipeline #46270 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants